Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

high-risk items pinpointer #13519

Merged
merged 4 commits into from
Sep 24, 2024
Merged

Conversation

simb11
Copy link
Contributor

@simb11 simb11 commented Aug 31, 2024

Описание изменений

Добавляет в аплинк трейтора пинпоинтер показывающий расстояние (но не направление!) до цели, стоит 2 ТК.
Был обновлён список possible_items_for_steal.
и ещё пофикшено то что нюкерский пинпоинтер не указывал на боеголовку.

Почему и что этот ПР улучшит

Решает проблему с тем что некоторые игроки прячут хайриски так, что без админской помощи и волшебного целеуказателя их не найти.

Пример

image

За деревом лежит золотой огнетушитель.

Авторство

Чеинжлог

🆑 Simbaka

  • add: В аплинк предателя добавлен особый целеуказатель, показывающий расстояние до ценного предмета (цели для кражи).
  • fix: Целеуказатель ядерных оперативников не указывал на боеголовку.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Fix labels Aug 31, 2024
@MrWestnow
Copy link
Contributor

@simb11
Бля идею спиздил и давно думал сделать такое, но ты ленивое решение осуществил
Во первых можно сделать наподобие такого
Otklik
Так что подключай какого нибудь рисовальщика и пусть 2д калякает

Второе принцип работы возьми только "холодно-горячо" без явного направления

Третье пусть работает условных 2 минуты с перегревом на 10 минут (опционально, но чтобы компенсировать 4-ый пункт)

Четвертое сделай бесплатным или 1 тк (вот увидишь никто не будет тратить половину бюджета на поинтер)

@TauKitty TauKitty requested a review from a team September 2, 2024 12:30
@simb11
Copy link
Contributor Author

simb11 commented Sep 2, 2024

переделал
Теперь этот пинпоинтер указывает лишь примерное расстояние до цели и не показывает направление к ней. Цена снижена до 2 ТК.

@ZER0-In
Copy link

ZER0-In commented Sep 5, 2024

переделал Теперь этот пинпоинтер указывает лишь примерное расстояние до цели и не показывает направление к ней. Цена снижена до 2 ТК.

  • Cлегка усложнить поиск тем, чтобы дать только три показателя по цвету? Красный далеко, желтый уже ближе и зеленый еще ближе. Шобы поиски посложнее шли. (Хотя и то что ща есть, сойдет).

  • Добавить звук, который издает пин, когда приближается очень близко к предмету? Например, чтобы триторы, что украли уже шмотку по этому звуку рядом понимали: "Бля, кто-то рядом пришел по мою душу и хочет отобрать у меня хайриск, нужно быть осторожнее". А не просто ревик-гиб головы без шансов от стелсочела, что надрачивает свой пин в руках и бегает рядом с грином другого тритора.

  • Добавить на вики в список предметов, которые ученые/сб могут опознать как нелегальную штуку, чтобы челы не бегали в открытую с ней по всей станции?

Я это все к тому, чтобы позволить челобасам выполнить свою цель, даже если их и опередили, но не обесценивать инициативу первых. Чтобы опоздавшим было сделать это не так просто. И чтобы не было моментов, когда тритор весь раунд сидит на жопе ровно, потом под самый шаттл покупает пин за 2тк и бегает с ревика гибает головы челобасам, собирая грин за чужой счет.

Comment on lines 264 to 265
itemlist = itemlist // To supress a 'variable defined but not used' error.
var/targetitem = input("Select item to search for.", "Item Mode Select","") as null|anything in itemlist::possible_items
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in itemlist::possible_items

это тут и ниже не должно работать, потому что possible_items - список, и до реальной инициализации объекта он не будет доступен. Списки просто особые и заполняются на скрытом шаге new объекта.

Можно те списки в глобалы переделать, и тут к глобальному списку просто обращаться.

Вместо possible_items_for_steal

#define ADD_TO_POIFS_LIST(type) ADD_TO_GLOBAL_LIST(type, possible_items_for_steal)
ADD_TO_POIFS_LIST(/obj/item/weapon/gun/energy/laser/selfcharging/captain)
ADD_TO_POIFS_LIST(/obj/item/weapon/hand_tele)
ADD_TO_POIFS_LIST(/obj/item/weapon/tank/jetpack/oxygen)
ADD_TO_POIFS_LIST(/obj/item/clothing/under/rank/captain)
ADD_TO_POIFS_LIST(/obj/item/device/aicard)
ADD_TO_POIFS_LIST(/obj/item/blueprints)
ADD_TO_POIFS_LIST(/obj/item/weapon/tank)
ADD_TO_POIFS_LIST(/obj/item/slime_extract)
ADD_TO_POIFS_LIST(/obj/item/weapon/reagent_containers/food/snacks/meat/corgi/ian)
ADD_TO_POIFS_LIST(/obj/item/clothing/under/rank/research_director)
ADD_TO_POIFS_LIST(/obj/item/clothing/under/rank/chief_engineer)
ADD_TO_POIFS_LIST(/obj/item/clothing/under/rank/chief_medical_officer)
ADD_TO_POIFS_LIST(/obj/item/clothing/under/rank/head_of_security)
ADD_TO_POIFS_LIST(/obj/item/clothing/under/rank/head_of_personnel)
ADD_TO_POIFS_LIST(/obj/item/weapon/reagent_containers/hypospray/cmo)
ADD_TO_POIFS_LIST(/obj/item/weapon/pinpointer)
ADD_TO_POIFS_LIST(/obj/item/weapon/reagent_containers/spray/extinguisher/golden)
ADD_TO_POIFS_LIST(/obj/item/weapon/gun/energy/gun/nuclear)
ADD_TO_POIFS_LIST(/obj/item/weapon/pickaxe/drill/diamond_drill)
ADD_TO_POIFS_LIST(/obj/item/weapon/storage/backpack/holding)
ADD_TO_POIFS_LIST(/obj/item/weapon/stock_parts/cell/hyper)
ADD_TO_POIFS_LIST(/obj/item/stack/sheet/mineral/diamond)
ADD_TO_POIFS_LIST(/obj/item/stack/sheet/mineral/gold)
ADD_TO_POIFS_LIST(/obj/item/stack/sheet/mineral/uranium)
#undef ADD_TO_POIFS_LIST

который не используется вообще и надо удалить

@Slegarrr
Copy link

Slegarrr commented Sep 8, 2024

Представили ролькохант-потенциал этой штуки в руках СБ? Уже вспоминаю ситуации, когда СБ затаривалось вещами из открытого аплинка или находило крутые приблуды в инвентаре трейтора и использовало их.

@simb11 simb11 requested a review from volas September 19, 2024 15:49
@volas volas merged commit 24f40c1 into TauCetiStation:master Sep 24, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Sep 24, 2024
@simb11 simb11 deleted the traitors-pinpointer branch September 25, 2024 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants